Skip to content

Update index.md #4566

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
doroudi wants to merge 3 commits into from
Closed

Update index.md #4566

doroudi wants to merge 3 commits into from

Conversation

@doroudi
Copy link

@doroudi doroudi commented Dec 27, 2024

Add YummyAdmin Project

Which change are you proposing?

  • Suggesting edits to an existing topic or collection
  • Curating a new topic or collection
  • Something that does not neatly fit into the binary options above

Editing an existing topic or collection

I'm suggesting these edits to an existing topic or collection:

  • Image (and my file is *.png, square, dimensions 288x288, size <= 75 kB)
  • Content (and my changes are in index.md)

Adding another Made in Iran Project to this collection

@doroudi
Update index.md
9aebd6a 
add yummy-admin
@doroudi doroudi requested a review from a team as a code owner December 27, 2024 05:11
@tomthorogood
Copy link
Contributor

tomthorogood commented Jan 10, 2025

@doroudi thanks for your contribution. It looks like you need to update to make the case of your entry match your username. 🫶

ahpook
ahpook reviewed Jan 25, 2025
View reviewed changes
Copy link
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will cause a test failure as-is, please fix with the suggestion.

@ahpook
Update collections/made-in-iran/index.md
3e8454d 
I'll go ahead and fix this.

Signed-off-by: Eric Sorenson <[email protected]>
ahpook
ahpook approved these changes Jan 25, 2025
View reviewed changes
Copy link
Contributor

@ahpook ahpook left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tests are passing now

@ahpook ahpook enabled auto-merge January 25, 2025 00:21
@davidmomin
Copy link

davidmomin commented Jan 25, 2025

can help me

@jmeridth
Copy link
Contributor

jmeridth commented Jan 26, 2025

🛑 Thank you for the contribution ❤️. Unfortunately, as per our contribution guidelines, we don't allow self promotion if you are a maintainer of the project:

Maintainers of a project cannot add a topic or collection for their own project. If a topic is popular enough to warrant inclusion, someone else will add or improve it.

We do this so that project get curated if there is enough support/activity in the community. Thank you!

@jmeridth jmeridth closed this Jan 26, 2025
auto-merge was automatically disabled January 26, 2025 00:09

Pull request was closed

@doroudi doroudi deleted the patch-1 branch January 28, 2025 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ahpook ahpook ahpook approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@doroudi @tomthorogood @davidmomin @jmeridth @ahpook